Skip to content
This repository was archived by the owner on Nov 10, 2025. It is now read-only.

docs: npm -> pnpm in README.md#172

Open
MichalBryxi wants to merge 1 commit intoembroider-build:mainfrom
MichalBryxi:patch-1
Open

docs: npm -> pnpm in README.md#172
MichalBryxi wants to merge 1 commit intoembroider-build:mainfrom
MichalBryxi:patch-1

Conversation

@MichalBryxi
Copy link
Copy Markdown
Contributor

  • Different technologies (npm / pnpm) is used
  • Align

Copy link
Copy Markdown
Contributor

@void-mAlex void-mAlex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

personally I disagree, wyou can use the app bp with either one of the two and this is a good hint to that effect

@MichalBryxi
Copy link
Copy Markdown
Contributor Author

MichalBryxi commented Apr 21, 2025

personally I disagree, wyou can use the app bp with either one of the two and this is a good hint to that effect

That would be IMO the case if both code examples would be listed side-by-side, for the same command and ideally for all commands.

Which is also a way and I can adjust the PR like that.

- Different technologies (npm / pnpm) is used
- Align
@MichalBryxi
Copy link
Copy Markdown
Contributor Author

Adjusted:

  • Aligned one more missed section
  • Added info box with tip/explanation

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants